Add dye and microscope search to Algolia autocomplete#347
Draft
tlambert03 wants to merge 2 commits intomainfrom
Draft
Add dye and microscope search to Algolia autocomplete#347tlambert03 wants to merge 2 commits intomainfrom
tlambert03 wants to merge 2 commits intomainfrom
Conversation
This commit implements issue #327 by adding dyes and microscopes to the comprehensive Algolia search bar. Backend changes: - Added DyeIndex and MicroscopeIndex to proteins/index.py - Added url property to Dye model (links to spectra viewer) - Added url and owner_name properties to Microscope model - Updated base.html template to expose dyeIndex and microscopeIndex Frontend changes: - Updated algolia.js to initialize dye and microscope indices - Added autocomplete sources for dyes (3 per page) and microscopes (2 per page) - Updated highlightHits to support manufacturer, part, slug, and owner_name fields - Added placeholder icon images for dye and microscope results Dyes link to the spectra viewer (/spectra/<slug>) and microscopes link to the microscope detail page (/microscope/<id>/). Note: Algolia indices need to be populated using Django management commands: python manage.py algolia_reindex --model Dye python manage.py algolia_reindex --model Microscope
for more information, see https://pre-commit.ci
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This commit closes #327 by adding dyes and microscopes to the comprehensive Algolia search bar.
Backend changes:
Frontend changes:
Dyes link to the spectra viewer (/spectra/) and microscopes link to the microscope detail page (/microscope//).
Note: Algolia indices need to be populated using Django management commands:
python manage.py algolia_reindex --model Dye
python manage.py algolia_reindex --model Microscope